Skip to content

fix(plugin): add explicit Agent tool warning against Bash CLI dispatch#189

Merged
onlycastle merged 2 commits intomainfrom
fix/agent-dispatch-bash-warning
Mar 26, 2026
Merged

fix(plugin): add explicit Agent tool warning against Bash CLI dispatch#189
onlycastle merged 2 commits intomainfrom
fix/agent-dispatch-bash-warning

Conversation

@onlycastle
Copy link
Copy Markdown
Owner

Summary

  • Skills (bp-analyze, bp-setup) now explicitly instruct the host to use the built-in Agent tool instead of shelling out to claude CLI for agent dispatch
  • Added open browser command to the GitHub star prompt flow in bp-setup
  • New test assertion verifying the anti-Bash dispatch contract language

Changes

  • 9cbd1e1 fix(plugin): add explicit Agent tool warning against Bash CLI dispatch
  • 1f26623 chore(plugin): rebuild dist artifacts

Test Plan

  • Code compiles without errors (tsc --noEmit clean)
  • All 8 plugin contract tests pass
  • Manual verification: run /bp-setup and confirm Agent dispatch works

Generated with Claude Code using /ship-it

onlycastle and others added 2 commits March 26, 2026 16:14
Skills now clearly instruct the host to use the built-in Agent tool
instead of shelling out to `claude` CLI for agent dispatch. Adds test
assertion for the new contract language.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@onlycastle
Copy link
Copy Markdown
Owner Author

Code Review

No issues found. All changes comply with project standards.

Reviewed:

  • CLAUDE.md compliance (language policy, no fallback policy) ✓
  • Shallow bug scan on diff ✓
  • Git history context (no regressions) ✓
  • Prior PR comment debt ✓
  • Code comment compliance ✓

Generated with Claude Code

If useful, react with a thumbs-up. Otherwise, thumbs-down.

@onlycastle onlycastle merged commit cfc07bc into main Mar 26, 2026
1 check passed
@onlycastle onlycastle deleted the fix/agent-dispatch-bash-warning branch March 26, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant